Skip to content

Add aria-current=page attribute to selected link#235

Open
oneiros wants to merge 1 commit into
codeplant:masterfrom
oneiros:aria-current
Open

Add aria-current=page attribute to selected link#235
oneiros wants to merge 1 commit into
codeplant:masterfrom
oneiros:aria-current

Conversation

@oneiros

@oneiros oneiros commented Jun 23, 2026

Copy link
Copy Markdown

This is my attempt to fix #234

I kept it deliberately simple. If you think this constitutes a breaking change, you might want to make it configurable instead.

Happy to add that or change anything you deem necessary.

(And thanks for this gem, have been using it for ages now.)

ClearlyClaire
ClearlyClaire previously approved these changes Jun 23, 2026
@andi andi dismissed ClearlyClaire’s stale review June 23, 2026 13:09

This seems to be a config issue. PRs should not be able to be approved by anyone.

@andi

andi commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Thanks, will review this asap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Active links need an aria-current attribute for accessibility

3 participants